Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Pokit Flow API Connect #81

Merged
merged 13 commits into from
Aug 8, 2024
Merged

feat: Pokit Flow API Connect #81

merged 13 commits into from
Aug 8, 2024

Conversation

stealmh
Copy link
Member

@stealmh stealmh commented Aug 7, 2024

#️⃣연관된 이슈

#76

📝작업 내용

  • 포킷 플로우에서 카테고리 관련된 모든 API를 연결했습니다.
    미분류 카테고리 조회는 서버쪽 작업중이라 아직 미완이고, 링크 추가 / 수정에서의 카테고리 추가 기능같은 경우에는 도형님 작업하실 때 겹쳐지는 화면이다보니 조율이 필요할 것 같아 일단 남겨두겠습니다.
  • Response body가 없는 경우가 있더라구요. 제가 깜빡하고 작업을 하지 않아 MoyaProvider+request에 해당 함수 추가해두었습니다.

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

도메인쪽 연결을 잘 사용하고 있는지 따끔하게 혼내주세요~~

close #76

stealmh added 12 commits August 7, 2024 21:30
@stealmh
[fix] 닉네임 텍스트필드 입력간 비활성 처리 추가
755d9bf
@stealmh
[chore] 페이지 쿼리 구조체 Equatable 추가
8339298
@stealmh
[chore] #76 포킷검색 api 접근제어자 및 필터링 추가
dd9f785
@stealmh
[feat] 닉네임 할당 누락 추가
67934b1
@stealmh
[feat] #76 카테고리 메인 조회 api 연결
6ab6b3c
@stealmh
[feat] #76 Body가 없을 때 요청 함수 추가
a21eddb
@stealmh
[chore] #76 삭제 / 수정 id 필드값 수정
a51f55b
@stealmh
[feat] #76 정렬 기능 inner action 이동 및 카테고리 삭제 API 연결
97808d5
@stealmh
[chore] 미사용 return 제거
f99ddff
@stealmh
[feat] #76 포킷 API 연결
485195c 
- 포킷 추가
- 포킷 수정
- 프로필 목록 조회
@stealmh
[feat] #76 포킷 추가 및 수정 시 내 포킷(카테고리)목록 API 연결
490d7b3
@stealmh
[feat] #76 카테고리 상세 화면 API 연결
a66701b
@stealmh stealmh self-assigned this Aug 7, 2024
ShapeKim98
ShapeKim98 approved these changes Aug 8, 2024
View reviewed changes
Copy link
Contributor

@ShapeKim98 ShapeKim98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다! 너무 잘 되네요! content쪽은 이번에 스키마가 좀 바껴서 반영해서 올리겠습니다!

Projects/Feature/FeatureCategoryDetail/Sources/CategoryDetailFeature.swift
@@ -164,9 +167,13 @@ private extension CategoryDetailFeature {

case .onAppear:
// - MARK: 목업 데이터 조회
state.domain.categoryListInQuiry = CategoryListInquiryResponse.mock.toDomain()
state.domain.contentList = ContentListInquiryResponse.mock.toDomain()
// state.domain.categoryListInQuiry = CategoryListInquiryResponse.mock.toDomain()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이건 이제 지워도 될것 같아요

Projects/Feature/FeatureCategoryDetail/Sources/CategoryDetailFeature.swift
return .run { send in
let request = BasePageableRequest(page: 0, size: 100, sort: ["desc"])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

size를 100으로 한 이유가 있나요?(페이징 처음이라 몰라서 물어보는것)

@ShapeKim98 ShapeKim98 merged commit 9599eb6 into develop Aug 8, 2024
1 check passed
@stealmh stealmh deleted the feat/#76-pokit-api branch September 3, 2024 12:26
@stealmh stealmh linked an issue Oct 2, 2024 that may be closed by this pull request
Pokit Flow API Connect #76
Closed
12 tasks
@stealmh stealmh added Feat 기능구현 API 🛜 기능구현 - API labels Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ShapeKim98 ShapeKim98 ShapeKim98 approved these changes

@ITlearning ITlearning Awaiting requested review from ITlearning

Assignees

@stealmh stealmh

Labels
API 🛜 기능구현 - API Feat 기능구현
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Pokit Flow API Connect
2 participants
@stealmh @ShapeKim98